Skip to content

docs: Update contribution guidelines for issue creation and feature p…#176

Merged
hannesrudolph merged 2 commits into
mainfrom
contributors2
May 7, 2025
Merged

docs: Update contribution guidelines for issue creation and feature p…#176
hannesrudolph merged 2 commits into
mainfrom
contributors2

Conversation

@hannesrudolph
Copy link
Copy Markdown
Collaborator

@hannesrudolph hannesrudolph commented May 7, 2025

…roposals


Important

Remove contributing.md and update sidebar to link to GitHub CONTRIBUTING.md.

  • Documentation:
    • Remove contributing.md file, previously containing contribution guidelines.
    • Update sidebars.ts to replace 'contributing' with a direct link to GitHub CONTRIBUTING.md.

This description was created by Ellipsis for c39e341. You can customize this summary. It will automatically update as commits are pushed.

@hannesrudolph hannesrudolph requested review from cte and mrubens as code owners May 7, 2025 02:25
@vercel
Copy link
Copy Markdown

vercel Bot commented May 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
roo-code-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 7, 2025 3:22am

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 7, 2025
Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to a4e5002 in 58 seconds. Click for details.
  • Reviewed 27 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 5 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. docs/contributing.md:101
  • Draft comment:
    Good improvement – the instructions now clearly specify using issue templates for bug reports and feature proposals. Consider adding hyperlinks directly to the templates if available for even easier access.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. docs/contributing.md:117
  • Draft comment:
    The restructuring of the 'Proposing New Features' section into a numbered list improves clarity. This layout better differentiates initial discussions from formal proposals.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. docs/contributing.md:101
  • Draft comment:
    Clear improvement: instructing contributors to use specific templates for issue creation enhances clarity. Consider adding direct links to the 'Bug Report' and 'Detailed Feature Proposal' templates if available.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
4. docs/contributing.md:117
  • Draft comment:
    The revised 'Proposing New Features' section (numbered steps for initial discussion and formal proposal) improves clarity. Optionally, add a brief note clarifying the difference between these stages.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
5. docs/contributing.md:145
  • Draft comment:
    There's a potential typographical error on line 145: 'roo-cline-.vsix'. If this is intended to refer to 'Roo Code', it might need to be corrected to 'roo-code-.vsix' (or the appropriate file name). Please double-check if this is a mistake.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_Ro5RqMtlvCg1Fp5q

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@dosubot dosubot Bot added the documentation Improvements or additions to documentation label May 7, 2025
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels May 7, 2025
Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed c39e341 in 56 seconds. Click for details.
  • Reviewed 232 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. sidebars.ts:131
  • Draft comment:
    Changed the sidebar entry from a simple doc reference to an external link. Ensure that the link URL 'https://github.com/RooVetGit/Roo-Code/blob/main/CONTRIBUTING.md' is correct and that this change is consistent with the redesign (i.e., verifying that the CONTRIBUTING.md file exists and is maintained). Optionally, consider adding a 'target' attribute if you want the link to open in a new tab.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 30% <= threshold 50% The comment is asking the PR author to ensure the correctness of a URL and its consistency with a redesign, which violates the rule against asking the author to confirm or ensure things. However, it also provides a suggestion about adding a target attribute, which is a valid code suggestion. The comment is partially useful but mostly violates the rules.
2. docs/contributing.md:1
  • Draft comment:
    Ensure deletion of this file is intentional—confirm that contribution guidelines have been fully migrated to CONTRIBUTING.md and that all references are updated.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.
3. sidebars.ts:133
  • Draft comment:
    Verify the new sidebar link correctly points to the updated CONTRIBUTING.md. The external link should match the intended documentation location.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_fpu1zxA8udobHbaG

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@hannesrudolph hannesrudolph merged commit 3208c29 into main May 7, 2025
3 checks passed
@hannesrudolph hannesrudolph deleted the contributors2 branch May 7, 2025 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant